Skip to content

Update pyproject.toml - #176

Merged
mikibonacci merged 1 commit into
aiidalab:mainfrom
danielhollas:update-pyproject
May 22, 2026
Merged

Update pyproject.toml#176
mikibonacci merged 1 commit into
aiidalab:mainfrom
danielhollas:update-pyproject

Conversation

@danielhollas

@danielhollas danielhollas commented May 19, 2026

Copy link
Copy Markdown
Contributor

Similar relaxation of version pin should probably be done for the muon plugin to prevent conflicts.

Comment thread pyproject.toml
"pre-commit",
"euphonic==1.3.0",
"aiida-vibroscopy~=1.3.0",
"aiida-phonopy~=1.2.1",

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pinning to minor versions is possibly still problematic, since any updates then need to be coordinated between all plugins at once (for common dependencies)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Plus we should aimed to introducing aiida-vibroscopy latest for the newest implementations

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@danielhollas I don't quite understand. Can you clarify what you're suggesting here? Note that there is complexity here in any case, as often AiiDA plugin developers move faster than AiiDAlab plugin developers.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fair enough, it's probably wise to pin to minor versions.

This PR is mostly about unpinning the patch versions.

- remove pre-commit from dependencies
- relax version pins to allow at least patch version updates
- add numpy as a build depedency for euphonic
@danielhollas
danielhollas marked this pull request as ready for review May 21, 2026 23:27
@danielhollas

Copy link
Copy Markdown
Contributor Author

CC @mikibonacci @cpignedoli this is ready for review

@mikibonacci mikibonacci left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! thanks @danielhollas
I'll also update the aiidalab-qe-muon plugin accordingly

@mikibonacci
mikibonacci merged commit c91cec7 into aiidalab:main May 22, 2026
1 check passed
@danielhollas
danielhollas deleted the update-pyproject branch May 22, 2026 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dependencies in pyproject.toml should not be pinned exactly

4 participants